Skip to content

GEOPY-2621: Remove out_group form ui.json. It's redundant, and was currently fai…#324

Merged
sebhmg merged 3 commits intorelease/ga-4.7from
GEOPY-2621
Dec 3, 2025
Merged

GEOPY-2621: Remove out_group form ui.json. It's redundant, and was currently fai…#324
sebhmg merged 3 commits intorelease/ga-4.7from
GEOPY-2621

Conversation

@benk-mira
Copy link
Contributor

@benk-mira benk-mira commented Dec 3, 2025

GEOPY-2621 - Tile estimation requires the same SimPEG group twice

@github-actions github-actions bot changed the title Remove out_group form ui.json. It's redundant, and was currently fai… GEOPY-2621: Remove out_group form ui.json. It's redundant, and was currently fai… Dec 3, 2025
@codecov
Copy link

codecov bot commented Dec 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.15%. Comparing base (57060f6) to head (5be13c2).
⚠️ Report is 4 commits behind head on release/ga-4.7.

Additional details and impacted files
@@               Coverage Diff               @@
##           release/ga-4.7     #324   +/-   ##
===============================================
  Coverage           91.15%   91.15%           
===============================================
  Files                 114      114           
  Lines                6197     6197           
  Branches              758      758           
===============================================
  Hits                 5649     5649           
  Misses                347      347           
  Partials              201      201           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@MatthieuCMira MatthieuCMira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I'm not against, but why can't we use the outgroup? I don't get it.

Copy link
Contributor

@MatthieuCMira MatthieuCMira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I'm not against, but why can't we use the outgroup? I don't get it.

@sebhmg sebhmg merged commit 7c35011 into release/ga-4.7 Dec 3, 2025
16 checks passed
@sebhmg sebhmg deleted the GEOPY-2621 branch December 3, 2025 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants